Chore/repo setup#1
Conversation
There was a problem hiding this comment.
Pull request overview
Initial repository scaffolding for the Routis monorepo, setting up a Next.js frontend, FastAPI backend, local Docker Compose environment, and CI workflows to support ongoing development.
Changes:
- Add baseline Next.js (apps/web) app with lint/build configuration and Docker image.
- Add baseline FastAPI (apps/api) service with uv-managed dependencies, Docker image, and a
/healthendpoint. - Add Docker Compose + CI workflow + repo templates/docs to standardize local/CI setup.
Reviewed changes
Copilot reviewed 26 out of 36 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Adds top-level project description, stack, and dev commands. |
| package.json | Introduces root-level script entry point(s). |
| docs/ARCHITECTURE.md | Aligns architecture doc with current Python version and stack table. |
| docker-compose.yml | Defines local dev stack (web, api, postgres, redis, backup stub). |
| apps/web/tsconfig.json | Adds TypeScript configuration for the web app. |
| apps/web/public/window.svg | Adds web static asset. |
| apps/web/public/vercel.svg | Adds web static asset. |
| apps/web/public/next.svg | Adds web static asset. |
| apps/web/public/globe.svg | Adds web static asset. |
| apps/web/public/file.svg | Adds web static asset. |
| apps/web/postcss.config.mjs | Configures PostCSS with Tailwind plugin. |
| apps/web/package.json | Defines web app dependencies and scripts. |
| apps/web/next.config.ts | Adds Next.js config placeholder. |
| apps/web/eslint.config.mjs | Adds ESLint flat config using Next presets/ignores. |
| apps/web/Dockerfile | Adds multi-stage Docker build for Next.js app. |
| apps/web/CLAUDE.md | Points to shared agent instructions file. |
| apps/web/app/page.tsx | Adds a minimal “Coming Soon” landing page. |
| apps/web/app/layout.tsx | Adds root layout, fonts, and metadata. |
| apps/web/app/globals.css | Adds Tailwind import and base styles. |
| apps/web/app/favicon.ico | Adds favicon asset. |
| apps/web/AGENTS.md | Adds agent guidance for working with the Next.js version used. |
| apps/web/.gitignore | Adds web-specific ignore rules. |
| apps/api/uv.lock | Adds locked Python dependency set for the API service. |
| apps/api/tests/test_main.py | Adds a placeholder test file. |
| apps/api/tests/init.py | Marks tests directory as a package. |
| apps/api/README.md | Adds API README placeholder file. |
| apps/api/pyproject.toml | Defines API project metadata, dependencies, and Ruff config. |
| apps/api/main.py | Adds FastAPI app with /health endpoint and uvicorn entrypoint. |
| apps/api/Dockerfile | Adds Docker image for API using uv sync + uvicorn. |
| apps/api/.python-version | Pins local Python version for the API. |
| .github/workflows/ci.yml | Adds CI jobs for web (npm) and api (uv/ruff/pytest + Postgres). |
| .github/pull_request_template.md | Adds PR template and checklist. |
| .github/ISSUE_TEMPLATE/feature.md | Adds feature issue template. |
| .github/ISSUE_TEMPLATE/bug.md | Adds bug issue template. |
| .env.example | Adds documented environment variable template for local/prod. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "name": "routis", | ||
| "private": true, | ||
| "scripts": { | ||
| "codegen": "cd apps/web && pnpm run codegen" |
There was a problem hiding this comment.
The root codegen script uses pnpm, but the repo’s web app and CI are using npm (apps/web/package-lock.json, CI runs npm ci). Also, apps/web/package.json doesn't define a codegen script, so this command will fail. Consider either adding a codegen script in apps/web and invoking it via npm --prefix apps/web run codegen, or removing/renaming this root script until codegen is implemented.
| @@ -0,0 +1,2 @@ | |||
| def test_placeholder(): | |||
| pass | |||
There was a problem hiding this comment.
This is a placeholder test that will always pass and doesn’t validate any behavior. Since CI runs pytest, it would be better to add at least one minimal assertion (e.g., calling GET /health via TestClient and asserting the JSON/status) or remove the test until real coverage is added.
| pass | |
| assert True |
|
|
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
No description provided.